Change SumoBot to deleted user

* In messaging, when a user who received a message is deleted
the sending users outbox will now show the message was to
deleted user
* When a message in your inbox is from a user who is deleted
it will appear as being from deleted user

Author: smithellis

kitsune/messages/handlers.py

@@ -13,8 +13,16 @@ def on_user_deletion(self, user: User) -> None:
         When a user is deleted:
         - Delete their outbox messages
         - Keep inbox messages for other users and reassign them to SumoBot
+        - Update outbox message recipients to SumoBot where the deleted user was a recipient
         """
         sumo_bot = Profile.get_sumo_bot()
         InboxMessage.objects.filter(to=user).delete()
         InboxMessage.objects.filter(sender=user).update(sender=sumo_bot)
+        
+        # Update outbox messages where the deleted user was a recipient
+        outbox_messages = OutboxMessage.objects.filter(to=user)
+        for message in outbox_messages:
+            message.to.remove(user)
+            message.to.add(sumo_bot)
+            
         OutboxMessage.objects.filter(sender=user).delete()

kitsune/messages/jinja2/messages/includes/macros.html

@@ -12,7 +12,11 @@
 
 {% macro name_link(user=None, name=None) -%}
   {% if user -%}
-    <a rel="nofollow" href="{{ profile_url(user) }}">{% if name %}{{ name }}{% else %}{{ display_name(user) }}{% endif %}</a>
+    {% if user.profile.is_system_account %}
+      {{ _('deleted user') }}
+    {% else %}
+      <a rel="nofollow" href="{{ profile_url(user) }}">{% if name %}{{ name }}{% else %}{{ display_name(user) }}{% endif %}</a>
+    {% endif %}
   {%- else -%}
     {{ name if name else _('None') }}
   {%- endif %}
@@ -42,7 +46,11 @@
       {{ avatar_link(message.sender, default_avatar) }}
     </div>
     <div class="user from">
+    {% if message.sender.profile.is_system_account %}
+      {{ _('deleted user') }}
+    {% else %}
       {{ name_link(message.sender) }} {{ datetimeformat(message.created) }}
+    {% endif %}
     </div>
   </div>
 </section>
@@ -55,9 +63,13 @@
   {% if message.recipients_count > 0 %}
     <p><strong>{{ _('To') }}:</strong>
     {% set comma = joiner(', ') %}
-    {% for user in message.to.all() -%}
+    {% for user in message.to_users -%}
         {{ comma() }}
+      {% if user.profile.is_system_account %}
+        {{ _('deleted user') }}
+      {% else %}
         {{ name_link(user) }}
+      {% endif %}
     {% else %}
       {{ name_link(message.recipient) }}
     {% endfor %}

kitsune/messages/jinja2/messages/outbox.html

@@ -38,9 +38,13 @@ <h1 class="sumo-page-heading">{{ title }}</h1>
                 <div class="email-cell sent">{{ datetimeformat(message.created) }}</div>
                 <div class="email-cell to">
                 {% for user in message.to.all()[:1] -%}
-                    <a rel="nofollow" href="{{ profile_url(user) }}">
+                    {% if user.profile.is_system_account %}
+                      {{ _('deleted user') }}
+                    {% else %}
+                      <a rel="nofollow" href="{{ profile_url(user) }}">
                         {{ user.profile.display_name }}
-                    </a>
+                      </a>
+                    {% endif %}
                     {%- if message.recipients_count > 1 -%}, ...{% endif %}
                 {%- endfor %}
                 </div>

kitsune/messages/views.py

@@ -26,7 +26,6 @@ def inbox(request):
         .prefetch_related("sender__profile")
     )
     count = messages.count()
-
     messages = paginate(request, messages, per_page=MESSAGES_PER_PAGE, count=count)
 
     return render(
@@ -42,19 +41,25 @@ def read(request, msgid):
     was_new = message.unread
     if was_new:
         message.update(read=True)
+
     initial = {"to": message.sender, "in_reply_to": message.pk}
     form = ReplyForm(initial=initial)
     response = render(
         request,
         "messages/read.html",
-        {"message": message, "form": form, "default_avatar": settings.DEFAULT_AVATAR},
+        {
+            "message": message,
+            "form": form,
+            "default_avatar": settings.DEFAULT_AVATAR,
+        },
     )
     return response
 
 
 @login_required
 def read_outbox(request, msgid):
     message = get_object_or_404(OutboxMessage, pk=msgid, sender=request.user)
+
     return render(
         request,
         "messages/read-outbox.html",
@@ -179,6 +184,26 @@ def preview_async(request):
 
 
 def _add_recipients(msg):
+    """Process and attach recipient information to a message object.
+
+    This helper function calculates recipient counts and assigns recipient-related
+    attributes to the message object for both individual users and groups.
+
+    Args:
+        msg: An OutboxMessage object to process.
+
+    Returns:
+        The modified message object with the following attributes set:
+            - recipients_count: Number of individual recipients
+            - to_groups_count: Number of group recipients
+            - recipient: The single recipient (if exactly one), else None
+            - recipient.is_bot_user: Boolean indicating if recipient is SumoBot
+            - to_groups: List of recipient groups with prefetched profiles
+
+    Note:
+        The function assumes msg.to and msg.to_group are valid related fields
+        on the message object.
+    """
     # Set the counts based on the lists
     msg.recipients_count = msg.to.all().count()
     msg.to_groups_count = msg.to_group.all().count()